Skip to content

Format Rust workspace and enforce rustfmt in CI - #105

Merged
leileizhang (lei9444) merged 1 commit into
mainfrom
lei9444/rustfmt-ci
Aug 20, 2026
Merged

Format Rust workspace and enforce rustfmt in CI#105
leileizhang (lei9444) merged 1 commit into
mainfrom
lei9444/rustfmt-ci

Conversation

@lei9444

Copy link
Copy Markdown
Contributor

Summary

  • apply cargo fmt --all to the remaining unformatted Rust workspace files
  • add a fast format CI job that runs cargo fmt --all -- --check
  • make the main test job depend on formatting and document the enforced command

The Rust source changes in this PR are formatting-only.

Validation

  • cargo fmt --all -- --check
  • cargo test -p dynwinrt --quiet (247 passed, 1 ignored)
  • cargo check -p static-bench-rust --quiet
  • git diff --check

Apply the existing rustfmt configuration across the workspace and add a fast CI gate with an actionable local command.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Mixed-language test coverage

Workflow status: ✅ Passed

Layer Lines Functions Branches/regions
Rust, including native .pyd/.node 82.01% 78.45% 81.95% regions
Generated Python projections 71.2% n/a 36.54% branches
JavaScript aggregate 18.92% 21.32% 52.69% branches
JavaScript runtime 19.91% 14.28% 7.14% branches
Generated WinRT projections 22.27% 20.02% 52.85% branches
Generated Classic COM projections 13.89% 25.06% 53.56% branches

View workflow run and download full HTML/LCOV/XML reports

@lei9444
leileizhang (lei9444) merged commit 65babcd into main Aug 20, 2026
26 checks passed
@lei9444
leileizhang (lei9444) deleted the lei9444/rustfmt-ci branch August 20, 2026 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants